Skip to content

03_max_of_three - #17

Open
stupnikovaliliya-maker wants to merge 2 commits into
PyLadiesCZ-Brno:mainfrom
stupnikovaliliya-maker:Liliia-03-max_of_three
Open

03_max_of_three#17
stupnikovaliliya-maker wants to merge 2 commits into
PyLadiesCZ-Brno:mainfrom
stupnikovaliliya-maker:Liliia-03-max_of_three

Conversation

@stupnikovaliliya-maker

@stupnikovaliliya-maker stupnikovaliliya-maker commented Oct 6, 2025

Copy link
Copy Markdown

Task_03 was modified — a new line return max(a, b, c) was added. All tests passed.

Fixes #16

Task_03 was modified — a new line return max(a, b, c) was added.
All tests passed.
@dslavicek

Copy link
Copy Markdown
Collaborator

@stupnikovaliliya-maker

  • test ti neprošel kvůli odsazení
  • autopep8 ti neprošel kvůli bílým znakům na prázdném řádku
  • prosím změň commit message tak, aby číslo tasku bylo od zbytku odděleno mezerou

@dslavicek

Copy link
Copy Markdown
Collaborator

@stupnikovaliliya-maker
Commity je podle zadání potřeba spojit do jednoho. Až je budeš spojovat změň prosím commit message tak, aby za číslem úkolu byla mezera. V commit messagích obecně oddělujeme slova mezerami, není tam k tomu potřeba používat podtržítka.

Spojení commitu jde udělat přes git rebase -i <hash commitu předcházejícího všem commitům, které chceš spojovat> (špičatý závorky tam nepatří, těma jen ukazuju kde má být hash). Následně se ti otevře textový editor s commitama a slovama pick u commitů. U commitu který chceš připojit k předchozímu přepíšeš pick na squash, uložíš a zavřeš editor. Potom se ti znova otevře textový editor, tentokrát bude chtít novou commit message. Napíšeš, uložíš, zavřeš a máš sqashnuto. Pushovat je pak potřeba s --force, protože sqashováním měníš historii.

Kdyžtak si řekni o pomoc někomu z koučů před lekcí.

@ZelenyMartin ZelenyMartin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideální je zde zkusit squashnout commity do jednoho, jak psal David.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

03

3 participants